wayland: Don't use g_error() on connection lost

When the Wayland compositor vanishes, all applications connected will
receive a SIGPIPE as soon as they try to use wl_display_dispatch().

Do not use g_error() to terminate the applications when this occurs,
g_error() means an error in the application while here it's not truly
the case.

Use g_warning() and exit() instead.

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
https://bugzilla.gnome.org/show_bug.cgi?id=745289

Updated Hungarian translation

Updated Hungarian translation

gdk: Use a better error message

Having "Assertion failed: success" on the console is not going to help
anyone.

Updated POTFILES.in and POTFILES.skip

Updated Polish translation

Updated Hebrew translation

GtkListBoxRow: add g_return_if_fail to grab_focus, don't crash

Although gtk_list_box_row_grab_focus() is not a public function
it can be easily called by gtk_widget_grab_focus() with a row argument
which has been removed from the list box and has box == NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=744879

Updated Greek translation

file chooser dialog: Prevent uneven button heights

The height of the text buttons depends on the font height,
whereas the search button has a fixed-size icon in it...
Prevent unevent heights by putting them all in a size group.

https://bugzilla.gnome.org/show_bug.cgi?id=745263

app chooser dialog: Prevent uneven button heights

The height of the text buttons depends on the font height,
whereas the search button has a fixed-size icon in it...
Prevent unevent heights by putting them all in a size group.

Add a configure check for new enough gettext

Check that xgettext understands glade files.

Stop building extract-strings

It is no longer needed.

Stop using extract-headers

Since 0.18.3, xgettext can extract strings from ui files.

Updated Slovak translation

Updated Greek translation

Updated Kazakh translation

printing: Always get details of Avahi printers via IPP

Request details of Avahi browsed printers even when
we've got their PPDs already. Some options are not
present in PPD.

https://bugzilla.gnome.org/show_bug.cgi?id=743323

printing: Get covers for each printer individually

Since we combine Avahi browsed printers and
printers of local CUPS in one backend we need to
get covers for each printer separately.

https://bugzilla.gnome.org/show_bug.cgi?id=743323

Updated Spanish translation

3.15.9

wayland: Don't allocate a full size SHM buffer when drawing using OpenGL

Before this patch, we'd always allocate a full size SHM buffer via
the wl_shm_pool, even though it would never be used. Instead allocate a
logical 1x1 cairo image surface.

https://bugzilla.gnome.org/show_bug.cgi?id=745076

wayland: Scale up wl_egl_window according to window scale

In order to support window scales for EGL windows, resize the
wl_egl_window to the window dimension multiplied with the window scale,
just as with SHM window buffers.

https://bugzilla.gnome.org/show_bug.cgi?id=745076

wayland: Properly support changing of surface scale

When the preferred surface scale changes, for example when entering a
wl_output with a higher scale than any previous entered output, recreate
the shm surface and redraw the window content with the new window scale.

Before this patch, the internal scale would be changed, but the shm
surface would not be recreated given the new scale, i.e. we'd attach a
buffer for a different scale than wl_surface.set_scale specified.

https://bugzilla.gnome.org/show_bug.cgi?id=745076

wayland: Don't ever try to set surface buffer scale when not supported

If the compositor is too old for handling surface buffer scales, never
tyr to set change it. This will effectively always leave it to its
initial state, i.e. 1.

https://bugzilla.gnome.org/show_bug.cgi?id=745076

gdkcairo: Bail if surface is in error

Don't try to paint onto an error surface. This happens for example when
gdk_cairo_set_source_pixbuf() is called with a pixbuf that is too big
for Cairo to handle.

Spotted by Christian Boxdörfer

css: Fix _gtk_bitmask_subtract()

We were doing the wrong thing *and* writing uninitialized memory while
doing so. BAD.

Also added tests exposing these.

https://bugzilla.redhat.com/show_bug.cgi?id=1185585

Updated Galician translations

Updated POTFILES.in

Updated Spanish translation

Updated Slovenian translation

Updated Slovenian translation

Updated Hebrew properties translation

scrolledwindow: Only start fade animation on "visible" scrollbars

If a scrollbar is not shown (because of policies, or because it isn't
necessary), it doesn't make sense to start fade animations on its window
on captured motion events.

Updated Hebrew translation

Fix the previous commit

The C_() macro only takes string literals.
Use g_dpgettext2() instead

Make gtk-im-context-none work

This was added a few years ago, as a way to have _no_ im context
at all. But it didn't actually work. Make it work, and streamline
the handling of none by moving it all to gtkimmodule.c.

As part of this, add context to the translated names of all
im modules we ship.

icontheme: allow directories with mtime 0

In order to provide a constant mtime between OS build and deploy time,
while also maintaining a hardlink content-addressed model independent of
timestamps, ostree sets all mtimes to 0.

The icon cache code currently ignores directories with mtime 0, assuming
they don't exist.

Track directory existence in a more precise way.

https://bugzilla.gnome.org/show_bug.cgi?id=745052

getting-started: move Drawing section to bottom.

Updated Spanish translation

icontheme: use desired size instead of negative for DIR_UNTHEMED SVGs

When loading SVGs from ICON_THEME_DIR_UNTHEMED GtkIconInfos,
such as those created for a GLoadableIcon, the size of the pixbuf to
load is set as a product of icon_info->scale.
But a few lines above, icon_info->scale is set to -1 for
ICON_THEME_DIR_UNTHEMED GtkIconInfos, so we'll end up always passing a
negative size to the GdkPixbuf loader, which is interpreted as the
nominal size of the image file.
Instead load the SVG at the desired scaled size in that case.

This fixes blurry icon in the notification panel in gnome-shell.

https://bugzilla.gnome.org/show_bug.cgi?id=744991

icontheme: fill correct information when loading GResource-backed GIcons

When loading a GResource-backed GFileIcon into a GtkIconInfo we
currently fail to populate the is_resource private field.
Also, since is_svg is set by looking at the filename, and
g_file_get_path() returns NULL for a GResourceFile, is_svg was always
FALSE.

https://bugzilla.gnome.org/show_bug.cgi?id=744991

Updated POTFILES.in

Getting-started: port drawing.c to GtkApplication

range: Make steppers use smooth autoscrolling

This makes them work much nicer than the old timer-based
implementation.

range: Make zoom scrolling less sensitive

Use the new long press delay factor to make the zoom
scrolling less easy to trigger accidentally.

long press: Add a delay-factor property

Add a way to set a delay factor that can be used
to make the long press more or less sensitive.
Making this a factor instead of exposing the delay
itself preserves the value of the setting as an
overall 'slow down long press' setting.

zoom scrolling: Improve the previous fix

As Sebastian pointed out, just resetting the initial slider
position was an incomplete fix, because it does not cause the
delta to be recomputed, which is important in this scenario,
because you've likely travelled some distance over the slider
before the long press kicks in.
Instead, explicitly record both the slider position and the
delta.

zoom scrolling: avoid 'jumping back'

We record the starting position on button press, but only
start the zoom mode when the long press timeout kicks in.
Depending on circumstances, this can cause a noticable jump.

Avoid this by resetting the recorded starting position after
the long press timeout. Suggested by Sebastian Keller.

Updated Chinese (Taiwan) translation

Updated Chinese (Taiwan) translation

Silence a compiler warning

Updated Hungarian translation

icon browser: Set up dnd from the detail popup

Allow dragging the various icon sizes as images.

icon browser: Add some dnd support

Use a custom tree model and enable dragging from the icon view.
Currently, we just support dropping the icon name as text.

icon browser: Activate on single-click

We want to get away from double-clicks everywhere.

icon browser: stop selecting

We don't need the selection here.

gdk/quartz: Fix mixup in attributes usage creating window implementation

First, attributes can be NULL (which is always the case when calling
gdk_window_ensure_native) so do not unconditionally dereference it.

Then the window_type should be taken directly from the GdkWindow as
in other backends (such as the X11 one for example).

https://bugzilla.gnome.org/show_bug.cgi?id=744942

Updated Slovak translation

statusicon: missed setting one uID

statusicon: fix crash storing pointer in UINT

https://bugzilla.gnome.org/show_bug.cgi?id=696505

Updated Chinese (Taiwan) translation

Updated Chinese (Taiwan) translation

Updated Greek translation

3.15.8

menubutton: Fix possible button/popover state inconsistences

While a popover is hiding, the modal grab is already gone and the toggle
button is clickable again, but clicking again at that time will result in
gtk_widget_show() trying to show an already shown widget (although fading
out and hidden soon) and the toggle button activated.

So let the menubutton set the active status only if the menu/popover
widget wasn't already shown, and ensure this doesn't get triggered by
double/triple button press events.

inspector: Destroy popovers on unmap

As an implementation detail, the popover hooks the fade out animation
on ::hide. Destroying the popover right away here is not a problem, but
prevents the animation from actually running. ::unmap will be run after
the animation is finished, so destroy the popover there.

popover: Add show/hide transitions

These have the same visual effect and timing than the gnome-shell ones.
During the hide animation, the popover has been made to take focus
elsewhere, and refuse to take any pointer/keyboard input until the popover
is shown again.

This has been based on work from Timm Bäder.

https://bugzilla.gnome.org/show_bug.cgi?id=741405

scrolled window: Avoid a crash

gtk_scrolled_window_allocate_scrollbar was calling
gtk_scrolled_window_allocate_child just to get the relative
allocation, overlooking the fact that that function is only
safe to call if the scrolled window _has_ a child. Unfortunately,
gtk_scrolled_window_allocate_scrollbar will sometimes get called
when that is not the case. Since we are really only interested
in the relative allocation, just get that directly. This
fixes a segfault in the style-properties-nth-child reftest.

scrolled window: Drop dead code

The allocation is never used in this function, so don't bother
getting it.

printing: Use correct units for margins taken via IPP

cups_printer_get_hard_margins() returned margins in millimeters
for margins taken via IPP request instead of Units used before.

dnd: Set the parent DnD wayland surface on gtk_drag_set_icon_window()

This is a better place to call this, both lets us remove all other calls,
and additionally handles the gtk_drag_set_icon_widget() entry point.

Do not return on void functions

pathbar: use the same parameter as in the declaration

docs: Add extra symbol links to input handling documentation

Cross-reference to the API documentation more consistently.

https://bugzilla.gnome.org/show_bug.cgi?id=744054

docs: Expand input handling documentation to cover event masks

Also try and clarify a few things about event propagation. Move
input-handling.xml into gtk-doc’s expand_content_files variable so it
automatically links to widget documentation. Add links from
gtk_widget_add_events() and friends to the new documentation.

https://bugzilla.gnome.org/show_bug.cgi?id=744054

docs: Mention events in the input handling documentation title

Make it a little easier to find these docs when searching for ‘event’.

https://bugzilla.gnome.org/show_bug.cgi?id=744054

gtk: Remove documentation references to the GTK_NO_WINDOW flag

It has been removed entirely, and replaced by
gtk_widget_get_has_window().

https://bugzilla.gnome.org/show_bug.cgi?id=744054

Updated Galician translations

color scale: simplify code a bit

It is not necessary to call gtk_widget_get_accessible repeatedly.

Explicitly set the a11y role of tooltip windows

This seems better than trying to identify the 'tooltipness'
of the window on the other side.

https://bugzilla.gnome.org/show_bug.cgi?id=744683

Drop nonworking code to set a11y roles of tooltips

This code relied on widget names, and we don't set those anymore.

widget factory: Don't make file chooser local-only

There's no reason to.

Updated Hungarian translation

build: Don't use shopt in the build command

shopt command only works in bash, so we should not use it in gtk/Makefile.am.

https://bugzilla.gnome.org/show_bug.cgi?id=743647

gtk: Fix various tiny typos in documentation comments

e.g. Invalid syntax in introspection annotations, typos in object names,
accidentally using a gtk-doc comment for an internal function.

docs: Remove non-existent symbols from gtk3-sections.txt

Eliminate a couple of gtk-doc warnings.

docs: Ignore generated files when building documentation

AppChooserDialog: Fix memleak

Updated Spanish translation

Updated Italian translation

wayland: Use display from GdkDragContext device

It will be useless to check the source window on the destination side,
it's at the moment always NULL. Fetch the display from the device instead,
which will be set for every GdkDragContext.

Updated Spanish translation

Updated Spanish translation

scrolledwindow: Ensure indicator windows are created at the right place

At the time of creating the indicator window, the scrollbar allocation is
poked and reused as the initial window dimensions. This usually happens
on two circumstances, either initially (so a ::size-allocate is emitted,
relocating the windows in the right places), or post-initialization when
calling set_overlay_scrolling() (so the scrollbars already have a valid
size allocation)

However, if the scrolledwindow is unrealized, and later re-realized again,
the scrollbars will already have a valid allocation, although 0,0 based
due to being contained in the previous indicator window. This comes out
wrong then, and the indicator window is given 0,0 based coordinates too.

Fix this by refactoring the scrollbar allocation code out of size_allocate,
and also use that given size at the time of creating the indicator windows,
this will provide the right widget-relative allocation anytime.

scrolledwindow: Fully reset indicators' state on unrealize

This avoids timers to run after realize(), and inconsistent initial states
if the scrolledwindow is just realized again.

Avoid a crash in the inspector under wayland

The Wayland-specific popover code in GtkWindow was not doing the
right thing with the separate display connection we use for the
inspector.

Do not use "return" when chaining up to a void function

Updated Italian translation

glarea: Better error handling

Currently, GtkGLArea will leak GError instances set during the context
creation, if an error is set.

If any error is set post-context creation, it should be displayed even
in the case a GL context exists; for instance, we can use the error
display facility for shader compilation errors.